Skip to content

fix: Don't find/replace FLAG_KEY in SDK instructions#193

Merged
dbolson merged 2 commits into
mainfrom
sc-239265/fix-flag-key-bug
Apr 22, 2024
Merged

fix: Don't find/replace FLAG_KEY in SDK instructions#193
dbolson merged 2 commits into
mainfrom
sc-239265/fix-flag-key-bug

Conversation

@dbolson
Copy link
Copy Markdown
Contributor

@dbolson dbolson commented Apr 19, 2024

This was breaking java and android examples.

Requirements

  • I have added test coverage for new or changed functionality
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions

Related issues

Provide links to any issues in this repository or elsewhere relating to this pull request.

Describe the solution you've provided

Provide a clear and concise description of what you expect to happen.

Describe alternatives you've considered

Provide a clear and concise description of any alternative solutions or features you've considered.

Additional context

Add any other context about the pull request here.

This was breaking java and android examples.
Comment thread internal/sdks/sdks.go
key,
"my-boolean-flag",
key,
"FLAG_KEY",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There aren't any "FLAG_KEY" values to replace with the new SDK instruction examples.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can get rid of all of these except for the ones above the comment line now that we've made them all the same value?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same thing with sdk key -- I forgot to clean that up when I removed the old instructions.

@dbolson dbolson merged commit 39ea1ce into main Apr 22, 2024
@dbolson dbolson deleted the sc-239265/fix-flag-key-bug branch April 22, 2024 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants